Skip to content

fix(ui): use dynamic provider options in spawn wizard (#816)#817

Open
Yaohua-Leo wants to merge 1 commit intoRightNow-AI:mainfrom
Yaohua-Leo:codex/fix-816-dynamic-spawn-providers
Open

fix(ui): use dynamic provider options in spawn wizard (#816)#817
Yaohua-Leo wants to merge 1 commit intoRightNow-AI:mainfrom
Yaohua-Leo:codex/fix-816-dynamic-spawn-providers

Conversation

@Yaohua-Leo
Copy link
Copy Markdown

Summary

Fixes #816 by replacing the spawn wizard's hardcoded provider dropdown with options loaded from /api/providers.

Changes

  • load provider options when opening the spawn wizard
  • render cloud/local provider groups dynamically from backend data
  • keep the currently selected provider visible as a fallback if the catalog has not loaded yet

Testing

ode --check crates/openfang-api/static/js/pages/agents.js

  • verified the spawn wizard template now renders spawnProviderGroups instead of hardcoded provider entries
  • Docker sandbox validation could not be run on this machine because the Docker daemon was not running

Impact

  • provider options in the spawn wizard now stay aligned with the backend model catalog
  • newly added providers no longer require manual frontend updates to appear in the UI

- load spawn wizard provider options from /api/providers
- render provider groups dynamically instead of hardcoded HTML
- keep the selected provider visible if catalog loading fails

Fixes RightNow-AI#816
@Yaohua-Leo
Copy link
Copy Markdown
Author

Docker validation is now complete.

Ran in node:22-bookworm:

  • node --check crates/openfang-api/static/js/pages/agents.js
  • verified crates/openfang-api/static/index_body.html renders spawnProviderGroups
  • verified crates/openfang-api/static/js/pages/agents.js loads provider data via loadSpawnProviders()
  • verified the old hardcoded spawn-wizard <option value="anthropic"> entry is no longer present

Result: all targeted checks passed.

@jaberjaber23
Copy link
Copy Markdown
Member

Reviewed and approved. This PR has merge conflicts with recent changes on main. Please rebase onto current main and we will merge it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Agent spawn wizard provider dropdown is still hardcoded and omits many supported providers

2 participants